Skip to content

Conversation

@josecm
Copy link
Member

@josecm josecm commented Nov 5, 2025

This PR adds support for the cortex-r52 based MPS3 support. See https://documentation-service.arm.com/static/615dbd31ac265639eac54439?token=.

We only tested this support using the QEMU version of the platform (https://www.qemu.org/docs/master/system/arm/mps2.html)

This PR is dependent on #265.

@danielRep
Copy link
Member

Can you solve the format issue @josecm ?

@danielRep danielRep self-assigned this Nov 5, 2025
@danielRep danielRep self-requested a review November 5, 2025 12:19
@josecm josecm force-pushed the feat/add_mps3_plat_support branch from 1b1381f to 829373e Compare November 5, 2025 12:26
@josecm
Copy link
Member Author

josecm commented Nov 5, 2025

Done @danielRep

Comment on lines 11 to 22
ptr_uart->state = 0x3;
ptr_uart->intstatus = 0xf;
}

void uart_enable(volatile struct cmsdk_uart_hw* ptr_uart)
{
ptr_uart->ctrl = 0x1;
}

void uart_putc(volatile struct cmsdk_uart_hw* ptr_uart, int8_t c)
{
while (ptr_uart->state & 0x1) { }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we use macros instead of magic numbers?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in force push

@josecm josecm force-pushed the feat/add_mps3_plat_support branch 4 times, most recently from 9f500d0 to 1e8fbb3 Compare November 5, 2025 13:28
@josecm josecm requested a review from sandro2pinto as a code owner November 5, 2025 13:28
@josecm josecm force-pushed the feat/add_mps3_plat_support branch 2 times, most recently from 6ff6589 to 9b67f2a Compare November 5, 2025 13:31
@josecm josecm force-pushed the feat/add_mps3_plat_support branch from 9b67f2a to fe1fc58 Compare November 5, 2025 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants